gtkdnd: Plug a leak

We create a cairo_t in order to find out the surface extents, but it
was being leaked, dragging the whole drag surface with it through
extra refs.

gtkmodelbutton: Plug a leak

The private struct had some data that needed freeing, but this widget
didn't have any destroy/dispose/finalize handlers.

gtkmenusectionbox: plug a leak

The list returned by gtk_container_get_children() wasn't being freed.

wayland: Plug some memleaks in DnD/clipboard code

gdk_atom_name() return value must be freed, this code was consistently
not doing so...

builder-tool: Don't strip h/vexpand

Setting these properties has the side-effect of setting the
corresponding -set properties, which makes GTK+ behave subtly
different. So don't mess with these.

GtkSearchBar: Don't vexpand

It turns out that setting vexpand to its default value has
side-effects that we need.

https://bugzilla.gnome.org/show_bug.cgi?id=751447

GtkImage: Make resource property work properly

gtk_image_set_from_resource was setting the resource_path,
only to have it overwritten again before returning. That
is not as it should be, so change things to set the resource
path late.

https://bugzilla.gnome.org/show_bug.cgi?id=751443

wayland: protect against unknown mimetype requests

This oddly can be reproduced with weston+weston-dnd, when dragging
anything from GTK+ into weston-dnd, it will insist on picking its
custom application/x-wayland-dnd-flower mimetype, and this request
forwarded by the compositor, even if GTK+ didn't announce it on
its wl_data_source mimetype list. (What should probably happen here
is that the request is silenced, and/or weston-dnd picks (null))

This should be harmless, we are leaking though the fd in that case,
because the emission of GdkEventSelection on an unhandled mimetype
results in NOP. In order to avoid this, we should check whether the
mimetype is supported at all on the backend code and possibly close
the fd, this involves storing these in the first place.

https://bugzilla.gnome.org/show_bug.cgi?id=751414

wayland: Ensure we close the fd on all error paths in data_source.send

https://bugzilla.gnome.org/show_bug.cgi?id=751414

wayland: close() the selection fd if we didn't start writing yet

If the other peer requests data too fast (too rare/unlikely though),
we might receive multiple gdk_wayland_selection_request_target() calls
with no ending gdk_wayland_selection_check_write(), in which case the
fd is leaked as no GOutputStream was created to take over it.

https://bugzilla.gnome.org/show_bug.cgi?id=751414

wayland: Ensure we cancel previous selection writes before starting one

We weren't catching all the places where the AsyncWriteData operation
should be cancelled, which could happen if we repeatedly request the
same target on different fds.

https://bugzilla.gnome.org/show_bug.cgi?id=751414

wayland: Do not close the descriptor on async_write_data_free()

At the moment we create the AsyncWriteData, the ownership of the
fd is granted to the GOutputStream, and the fd set to -1, so at
this moment we're just silently getting EBADFD.

This partially reverts 25885ca600fff, the initialization of .fd
to -1 is valid and stays though.

https://bugzilla.gnome.org/show_bug.cgi?id=751414

Updated Hungarian translation

wayland: Ensure device grabs generate crossing/focus events

On X11 this is something the windowing system does for us, which the
wayland backend should emulate, being grabs completely client-side.

So, if the grab and current focus windows differ, make sure we emit
focus/crossing events as it corresponds to the grab device.

wayland: Update grab serial when ungrabbing keyboards

This was being done so only on pointers. Internally, a GdkDeviceGrabInfo
is kept for each of the master pointer/keyboard, failing to do this for
keyboards results in a stuck keyboard grab.

https://bugzilla.gnome.org/show_bug.cgi?id=748892

wayland: Properly initialize/finalize where we store the data_source.send fd

The fd must be closed on async_write_data_free(), but we should also
initialize it to -1 so gdk_wayland_selection_check_write() doesn't wrongly
pick the stdin fd.

https://bugzilla.gnome.org/show_bug.cgi?id=751414

GtkWindow: some min/nat size corrections.

Don't add the container border to the title request size; it
is only used for the child widget.

Don't call gtk_widget_get_preferred_width_for_height() for
the title bar with an unrelated height and subtract the title
bar height before querying the child widget width.

Guard against negative size requests after substracting the
borders/shadows and the title bar.

https://bugzilla.gnome.org/show_bug.cgi?id=751341

3.17.4

Fix up doc comments

Fix up expected results of a11y tests

These changed due to th extra box that GtkAssistant now
inserts for each page.

Fix an oversight

GtkPlacesSidebar: Simplify sidebar code

Make this code follow GTK+ styles and conventions better.

GktPlacesSidebar: Plug a ton of leaks

If you don't keep in mind that g_object_get() returns references to
objects and copies fo strings, things leak all over the place.

sidebarrow: Avoid object-valued properties

They tend to leak...as these do.

render: Fix ridge/groove border rendering

The typo strikes again!

GtkHeaderBar: reduce minimum title width

The minimum title width affects the minimum window width
for CSD windows. To allow smaller windows like without
CSD reduce it a bit (276px vs 156px min width)

https://bugzilla.gnome.org/show_bug.cgi?id=751341

GtkWindow: fix default empty window size with CSD

In the non-CSD case we checked for 0x0 window size requisition
and replaced it with 200x200 so the window was still visible.
This no longer works in case of CSD as the shadow and title bar
are always added to the requisition resulting in a titlebar/shadow
only window in case there is no child widget (this is currently
visible under wayland or when setting GTK_CSD=1).

Instead of special casing the final window size, special case
the child requisition paths instead. This gives us the same
requisition in both, CSD and non-CSD cases (the header bar
has a too large minimum width atm so the resulting window is
still not the same)

https://bugzilla.gnome.org/show_bug.cgi?id=751341

mir: fix toggling between maximized and restored

mir: fix wheel scrolling in both directions

mir: emit GDK_ENTER_NOTIFY on pointer enter

mir: remove non-useful logging

mir: don't need dummy egl surface

mir: emit configure event on window move/resize

Without this, some windows don't restart the paint clock needed to
actually render the contents.

mir: fix build due to missing api

Adwaita: scale marks visible on the dark variant

Updated Hungarian translation

GtkScale: More work on mark spacing

Try harder not to push the scale out of balance by marks. With
this change, the remaining difference in the drawing is down
to asymmetric assets and margins coming out of the theme.
To fully support such asymmetric rendering, we need implement
baseline alignment for scales.

https://bugzilla.gnome.org/show_bug.cgi?id=749650

printing: Get output bin via IPP

Request "output-bin-supported" and "output-bin-default" attributes through
IPP if there is no PPD for selected printer.
Pass "output-bin" option with other options in printer_get_options().
Translate standard IPP values of "output-bin" option.

https://bugzilla.gnome.org/show_bug.cgi?id=725441

GtkStack: Avoid some unnecessary complication

We already have an in-destruction flag, so use it.

Fix calculation of edge input window size in case the real window is too small.

This error resulted in warnings like
"pixman_region32_init_rect: Invalid rectangle passed"

In case the window is smaller than handle_size * 2 the resulting
edge window got a negative size. Prevent that by limiting the
handle size to half the respective edge length. This also
prevents the corner windows from overlapping in case the window
is too small.

Revert "window: Check if we can use CSD before enabling them"

This reverts commit c5e5ee67490e7e7af56052d8f8beb75db002c2f1.

It turns out that this commit breaks the default sizing of GtkWindow,
which in turn causes the window to be smaller than the handle size,
which ends up sending invalid rectangles to Cairo.

We'll have to find another way to fix the CSD issue.

See also: https://bugzilla.gnome.org/show_bug.cgi?id=751140

https://bugzilla.gnome.org/show_bug.cgi?id=750343

Avoid spurious operations on destroy

When the stack is destroyed we do not want to waste time running
animations and notifying listeners about which is our current
visible child.

This is not only an optimization, but it is important for the stack
switcher widgets: since they are in another branch of the hieratchy
we do not want to get notifications while the stack is being destroyed.

Based on a patch by Paolo Borelli
https://bugzilla.gnome.org/show_bug.cgi?id=724506

Pacify gobject-introspection

gobject-introspection complains about the parameter names of
vfuncs. Why, you ask ? No idea.

GtkRange: Improve zoom handling a bit

The calculation to update the initial slider position on zoom
changes was not working correctly when using keys to toggle
zoom on and off for scales. Avoid it by updating the position
beforehand.

GtkRange: Make drag gesture less agressive

Moving the mouse while pressing one of the steppers was making
the slider jump to the end, unexpectedly. This was caused by the
drag gesture kicking in when it shouldn't. Fix this by making
all drag gesture signal handlers only do something if we are in
a drag thats started on the slider.

https://bugzilla.gnome.org/show_bug.cgi?id=751050

file chooser: Avoid tall combos

The filter combo box in the file chooser was stretching to match
the height of the extra widgets next to it. Since, stretched combo
boxes are not a pretty sight, make it stick to the top instead.

Patch by Caolán McNamara,

https://bugzilla.gnome.org/show_bug.cgi?id=751227

GtkOverlay: Sync up parameter names

gtk-doc gets confused if the parameter names between header,
source and doc comment don't match up.

Fix a typo

gtk-doc complained about a multi-line Since: tag, but the
real culprit was a typo in the Deprecated: tag.

Tell gtk-doc to ignore private headers

Tell gtk-doc to ignore new private headers as well, otherwise
it complains about undocumented symbols in them.

GtkWindow: Small documentation addition

GtkTextView: Small documentation addition

Fix up index for some 3.14 symbols

Marking these as "Since: 3.14.1" may be more accurate, but
it causes gtk-doc to not put these symbols in any versioned
index at all (it generates an unused index for 3.14.1). So,
lets lie a little, and say these symbols were added in 3.14.

gtk-demo: Add forgotten source file

The font_features.c file was not added as a resource, so that
gtk3-demo could not display it.

gtk-demo: Many cosmetic fixes

Mostly update window titles, and some addition to descriptions.

GtkAssistant: Fix criticals on dispose

After the recent change thta introduced boxes between the pages
and the notebook, we were no longer careful enough when disposing
the assistant. Fix that up.

gtk3-demo: Improve the overlay demo

Add some more information to the description.

gtk-demo: A better overlay demo

This shows the new passthrough feature.

Updated German translation

Updated POTFILES.skip

gtk-demo: Don't Distribute demos.h

Since demos.h is now generated according to the platform for which GTK+ is
built, don't distribute it. Generate a Windows-specific demos.h.win32 and
distribute that instead, in which the Visual Studio build files will copy
it to demos.h, so that the build will proceed normally.

https://bugzilla.gnome.org/show_bug.cgi?id=749622

Bump Pango requirement

The font features demo that was just added uses bleeding-edge
pango features.

Add a font features demo

This started life as a standalone tool, but it is nicer
to have it integrated in gtk3-demo.

file chooser: Small cleanup

Ask the search model if it is empty, instead of maintaining this
information manually in the file chooser widget.

GtkSearchEngineTracker: Fix SPARQL syntax

The function is called tracker:uri-is-descendant, not
tracker-uri-is-descendant.

Forgotten file

GtkSearchEngineSimple: Avoid a private struct

We can just make this type final, and avoid the private struct.

GtkSearchEngineTracker: Avoid a private struct

We can just make this type final, and avoid the private struct.

Avoid an unnecessary check

We create the indexed_locations array unconditionally.

GtkSearchEngine: Avoid a crash

Add a destroy notify for the data of the callback, so we don't
end up leaving a dangling pointer behind for a short while if
the native engine is finalized before the simple one. This
was showing up as crash when typing and backspacing in the
search entry of the file chooser.

GtkSearchEngine: Avoid crawling indexed locations

Implement the IsIndexed callback for tracker. This requires
reading settings of the tracker file miner. We are careful
to avoid a hard dependency on the tracker schemas.

GtkSearchEngine: Avoid crawling indexed locations

Add a framework to the simple engine that allows to skip
locations which are indexed by the native engine.

GtkSearchEngine: Initialize recursive flag

Otherwise all our searches become non-recursive, which was not
the intention.

GtkSearchEngine: Add recursive flag

Add a flag for recursive search, and implement non-recursive
search in both the tracker and simple search engines.

This is not currently used in the file chooser.

Remove an unused field

GtkSearchEngine: Remove unused code

Nothing ever uses or emits the hits-subtracted signal.
Nautilus dropped it a few years ago. Time to follow suit.

Use infos from search engine instead of getting them again

When the search engine provides hits with GFileInfo, use that
to add the hits to the model directly, without going through
another round of async get_info calls.

To do this, we add a batched variant of the
_gtk_file_system_model_update_file call that takes lists of
GFiles and GFileInfos. Again, we can avoid repeated resorting
that happens when the files are updated individually.

Fix an oversight in the previous commit

We lost the line marking the search model as non-empty, causing
us to show the 'no results' page despite there being results.

Request the right attributes in the simple search engine

This change makes it so that the file info passed along
from the simple search engine has all the attributes that
the file system model wants.

search engine: Pass file infos along for hits

file chooser: Use batched insertion for search results

This helps avoiding excess work when procesing many search
results, as typically happens with short search strings
and recursive search.

https://bugzilla.gnome.org/show_bug.cgi?id=751169

file system model: Add batched insertion

Add a batched version of gtk_file_system_model_add_and_query_file
that takes a list of files and avoids resorting the model for each
individual insertion. The querying is still done one-file-at-a-time,
so more optimization is certainly possible.

Revert "window: Reinstate logic for should_use_csd()"

This reverts commit f043fd532dc66a7abb1d4ea57c666655330cc429.

The whole point of having can_use_csd() return TRUE when GTK_CSD is
unset is to use it inside enable_csd().

https://bugzilla.gnome.org/show_bug.cgi?id=751140

window: Reinstate logic for should_use_csd()

The old should_use_csd() function would return FALSE if the GTK_CSD
environment variable is unset; the change in commit c5e5ee67490e7e7
made it return TRUE if GTK_CSD is unset. This has a cascade effect
on the window size, which causes invalid rectangles to bubble down
to Pixman.

https://bugzilla.gnome.org/show_bug.cgi?id=751140

gtkdialog: Remove 2px border

messagedialog: remove 2px border

filechooserdialog: Remove space between window and content

Was removed in 232d6f84f0495e9effedc16d700fc13f5a6e302.

Adwaita: indentation

- lapo was shouting at me.

Adwaita: volume button and scale button padding

I was assuming that GtkVolumeButton and GtkScaleButton could only
contain a icon, now they get the image-button padding by default,
but if the text-button class is applied it is not overridden.

Adwaita: scale popup quick fix

Adwaita: markup cleanup

- move touch selection selectors to its own section
- use scss style markup for .button common elsewhere
  (more compact and legible)

HC: touch text selection styling

- sync with Adwaita
- try to avoid using image assets

https://bugzilla.gnome.org/show_bug.cgi?id=750883

Fix return value error in _gtk_window_get_popover_parent

Cosmetic change

GtkPlacesSidebar: Fix rtl appearance

Apply the necessary directional variants for asymmetric
css padding.

https://bugzilla.gnome.org/show_bug.cgi?id=751084

overlay: remove gtk_overlay_add_pass_through_overlay()

The API to access this functionality will be the setter we just added in
the previous commit.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

overlay: add setters and getters for pass-through child property

This will make the API easier to use from bindings too.

https://bugzilla.gnome.org/show_bug.cgi?id=750568

docs: add new GtkOverlay methods to gtk3-sections.txt

https://bugzilla.gnome.org/show_bug.cgi?id=750568

configure: test have_docbook_style != yes, not have_docbook_dtd

file chooser: Drop unused variables

file chooser: Use a revealer

Use a revealer to manage the visibility of the header area where
we show the pathbar, the location entry or search. This is a bit
smoother, and makes search more similar to a search bar.